Skip to content

Conversation

@Will-ShaoHua
Copy link
Contributor

@Will-ShaoHua Will-ShaoHua commented Sep 10, 2025

Problem

the debug string is missed somehow and not being logged

let msg = indent(
                `codewhisperer: request-id: ${requestId},
                timestamp(epoch): ${Date.now()},
                timezone: ${timezone},
                datetime: ${new Date().toLocaleString([], { timeZone: timezone })},
                vscode version: '${vscode.version}',
                extension version: '${extensionVersion}',
                filename: '${EditorContext.getFileName(editor)}',
                left context of line:  '${session.leftContextOfCurrentLine}',
                line number: ${session.startPos.line},
                character location: ${session.startPos.character},
                latency: ${latency} ms.
                Recommendations:`,
                4,
                true
            ).trimStart()

Solution


  • Treat all work as PUBLIC. Private feature/x branches will not be squash-merged at release time.
  • Your code changes must meet the guidelines in CONTRIBUTING.md.
  • License: I confirm that my contribution is made under the terms of the Apache 2.0 license.

@github-actions
Copy link

  • This pull request modifies code in src/* but no tests were added/updated.
    • Confirm whether tests should be added or ensure the PR description explains why tests are not required.

@Will-ShaoHua Will-ShaoHua changed the title chore: add inline debug logs performance: add inline debug logs Sep 10, 2025
@Will-ShaoHua Will-ShaoHua changed the title performance: add inline debug logs perf: add inline debug logs Sep 10, 2025
@Will-ShaoHua Will-ShaoHua reopened this Sep 10, 2025
@Will-ShaoHua Will-ShaoHua removed the request for review from leigaol September 10, 2025 19:58
@leigaol leigaol merged commit 70b6101 into aws:feature/inline-rollback Sep 10, 2025
27 of 29 checks passed
@Will-ShaoHua Will-ShaoHua deleted the inline branch September 10, 2025 20:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants